@tigrannajaryan I take for granted #363 is not a hard requirement for this?

@tigrannajaryan I take for granted #363 is not a hard requirement for this?

No, it is not because we prohibit using enum value names in JSON. We require using numeric values.

@tigrannajaryan this should wait first for the specification to be released per our guidance correct? Anything that is specified in the specification repo becomes the rule once it is released.

Once we are ready (both PRs have enough approvals) we can merge both PRs at the same time.

Depends on open-telemetry/opentelemetry-specification#3040

After open-telemetry/opentelemetry-specification#2930 is merged the "protocol" part of the protobufs is stable, but what about the helpers (flag values (or mask values, unclear though)) that are not defined by the protocol, are these stable? What does it mean for JSON encoding?

Update: If we keep them, what are the guarantees for these fields since they are not put on the wire, so definitely they will always be "wire compatible".

After open-telemetry/opentelemetry-specification#2930 is merged the "protocol" part of the protobufs is stable, but what about the helpers (flag values (or mask values, unclear though)) that are not defined by the protocol, are these stable? What does it mean for JSON encoding?

Helpers are not visible anywhere on the wire (we intentionally disallowed enum values as strings to make sure enum names don't end on wire), so I think the fact that we have helpers or don't have helpers has no impact for JSON encoding. Do you see this differently?

After open-telemetry/opentelemetry-specification#2930 is merged the "protocol" part of the protobufs is stable, but what about the helpers (flag values (or mask values, unclear though)) that are not defined by the protocol, are these stable? What does it mean for JSON encoding?

Helpers are not visible anywhere on the wire (we intentionally disallowed enum values as strings to make sure enum names don't end on wire), so I think the fact that we have helpers or don't have helpers has no impact for JSON encoding. Do you see this differently?

@bogdandrutu does this answer your question? Any other concerns or we can move forward with this?

This is ready to be merged after 1.18 spec release.

Merging as 1.18.0 was released and open-telemetry/opentelemetry-specification#2930 was merged.